Skip to content

[Fix] NanoGPT Muse Spark fails during tool use - #1310

Merged
taltas merged 2 commits into
mainfrom
fix/nanogpt-muse-tool-calls-0h47xbcviys51
Aug 22, 2026
Merged

[Fix] NanoGPT Muse Spark fails during tool use#1310
taltas merged 2 commits into
mainfrom
fix/nanogpt-muse-tool-calls-0h47xbcviys51

Conversation

@zoomote

@zoomote zoomote Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

​Opened on behalf of @navedmerchant. Follow up by mentioning @roomote, in the web UI, or in Discord.

What changed

NanoGPT requests for meta/muse-spark-1.2-contributor now keep mixed tool-result and environment context in one contiguous tool message when replaying multi-turn history. A focused regression test reproduces repeated tool-call/result cycles from the diagnostics while confirming that parallel tool calls remain enabled.

Why this change was made

The diagnostics showed four valid, sequential tool calls before NanoGPT rejected the fifth response with Upstream emitted malformed tool call data that could not be repaired. Each tool result was followed by environment text that Zoo previously replayed as a separate user message, so this compatibility mitigation preserves that context on the matching tool message instead of changing unrelated parallel-call behavior.

Impact

Muse Spark 1.2 Contributor receives a more stable multi-turn tool history without losing environment context or parallel tool support. Other NanoGPT models keep their existing message conversion.

Related PRs

@codecov

codecov Bot commented Aug 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@taltas
taltas marked this pull request as ready for review August 22, 2026 14:38
@taltas
taltas enabled auto-merge August 22, 2026 14:38
@taltas
taltas added this pull request to the merge queue Aug 22, 2026
Merged via the queue into main with commit 7e85e27 Aug 22, 2026
24 checks passed
@taltas
taltas deleted the fix/nanogpt-muse-tool-calls-0h47xbcviys51 branch August 22, 2026 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants